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Replace Type Code with Polymorphism 

You have a type code that affects the behavior of a class. 

Replace the type code with classes: one for each type code variant. 


Rigid Mountain Bike 




price 

off_road_ability 



Mountain Bike 




off_road_ability 

price 





Front Suspension 
Mountain Bike 




price 

off_road_ability 



Full Suspension 
Mountain Bike 




price 

off_road_ability 




Motivation 

This situation is usually indicated by the presence of case-like conditional state- 
ments. These may be case statements or if-then-else constructs. In both forms 
they test the value of the type code and then execute different code depending 
on the value of the type code. 

Removing Conditional Logic 

There are three different refactorings to consider when you’re trying to remove 
conditional logic: Replace Type Code with Polymorphism, Replace Type Code 
with Module Extension, or Replace Type Code with State/Strategy. The choice 
depends on relatively subtle design differences. 

If the methods that use the type code make up a large portion of the class, I use 
Replace Type Code with Polymorphism. It’s the simplest, and just takes advan- 
tage of Ruby’s duck-typing to remove the conditional statements. It involves 
blowing away the original class and replacing it with a new class for each type 
code. Since the original class was heavily reliant on the type code, it generally 
makes sense for the clients of the original class to construct an instance of one 



Replace 
Type Code 
with 

Polymorphism 



Download at WoweBook.Com 



Chapter 8 Organizing Data 


of the new type classes (because they were probably injecting the type into the 
original class anyway). 

If the class has a large chunk of behavior that doesn’t use the type code, I 
choose either Replace Type Code with Module Extension or Replace Type Code 
with State/Strategy. These have the advantage of enabling me to change the type 
at runtime. In the former we extend a module, mixing in the module’s behavior 
onto the object. Instance variables are shared automatically between the object 
and the module, which can simplify things. Replace Type Code with State/Strat- 
egy uses delegation: The parent object delegates to the state object for state-spe- 
cific behavior. The state object can be swapped out at runtime when a change 
in behavior is required. Because of the delegation, sharing of instance variables 
between the parent object and the state object can be awkward. So the question 
becomes, why would you choose State/Strategy over Module Extension? It turns 
out that you can’t unmix a module in Ruby, so removing undesired behavior 
can be difficult. When the state changes become complex enough that unwanted 
behavior cannot be removed or overridden, I choose Replace Type Code with 
State/Strategy. 

The great thing about Ruby is that you can do Replace Type Code with Poly- 
morphism without inheritance or implementing an interface, something that is 
impossible in a language such as Java or C#. 


Mechanics 


1. Create a class to represent each type code variant. 

2. Change the class that uses the type code into a module. Include the mod- 
ule into each of the new type classes. 

3. Change the callers of the original class to create an instance of the desired 
type instead. 
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4. Test. 

5. Choose one of the methods that use the type code. Override the method 
on one of the type classes, 

6. Test. 


7. Do the same for the other type classes, removing the method on the mod- 
ule when you’re done. 
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8. Test. 

9. Repeat for the other methods that use the type code. 

10. Test. 

11. Remove the module if it no longer houses useful behavior. 


Example 

For this case Tm modeling mountain bikes. An instance of the MountainBike can 
either be ; rigid (having no suspension), ;front_suspension, or :full_suspension (having 
both front and rear suspension). The @type_code determines how things like off_ 
road_ability and price are calculated: 

class MountainBike. . . 
def initialize(params) 

params.each { |key, value | instance_variable_set "@#{key}", value } 
end 

def off_road_ability 

result = @tire_width * TIRE_WIDTH_FACTOR 

if @type_code == :front_suspension || @type_code == :ful1_suspension 
result += @front_fork_trave1 * FRONT_SUSPENSION_FACTOR 
end 

if @type_code == :full_suspension 
result += @rear_fork_travel * REAR_SUSPENSION_FACTOR 
end 

result 

end 

def price 
case @type_code 
when : rigid 

(1 + @commission) * @base_price 
when :front_suspension 

(1 + ©commission) * @base_price + @front_suspension_price 
when :full_suspension 

(1 + ©commission) * ©base_price + ©front_suspension_price + 
©rear_suspensi on_pri ce 
end 
end 
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It can be used like this: 

bike = MountainBike.new(:type_code => : rigid, :tire_width => 2.5) 
bikeZ = MountainBike.new(:type_code => :front_suspension, :tire_width => 2, 
:front_fork_trave1 => 3) 

We’ll start by creating a class for each type. We’ll change MountainBike to a mod- 
ule, and include it in each of our new classes. 

class RigidMountainBike 
include MountainBike 
end 

class FrontSuspensi onMountai nBi ke 
include MountainBike 
end 

class Ful 1 Suspensi onMountai nBi ke 
include MountainBike 
end 


class module MountainBike... 


def wheeLcircumference 
Math::PI * (@wheel_diameter + @tire_diameter) 
end 
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def off_road_ability 
result = @tire_width * TIRE_WIDTH_FACTOR 

if @type_code == :front_suspension || @type_code == :full_suspension 
result += @front_fork_travel * FRONT_SUSPENSION_FACTOR 
end 

if @type_code == :full_suspension 
result += @rear_fork_travel * REAR_SUSPENSION_FACTOR 
end 
result 
end 

def price 
case @type_code 
when : rigid 

(1 + ©commission) * @base_price 
when :front_suspension 

(1 + ©commission) * ©base_price + ©front_suspension_price 
when :full_suspension 
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(1 + @commission) * @base_price + @front_suspension_price + 
@rear_suspensi on_pri ce 
end 
end 
end 

The callers will need to change to create our new type. 

bike = MountainBike.new(:type_code => :front_suspension, :tire_width => 2, 
:front_fork_travel => 3) 

becomes 

bike = FrontSuspensionMountainBike.new(:type_code => :front_suspension, 
:tire_width => 2, 

:front_fork_travel => 3) 

Although we haven’t gotten far, we should he able to run the tests and they 
should still pass. 

Next we use Replace Conditional with Polymorphism on one of the methods 
that we want to call polymorphically, overriding it for one of our new classes. I 
choose price and start with RigidMountainBike. 

class RigidMountainBike 
include MountainBike 


def price 

(1 + ©commission) * @base_price 
end 
end 

This new method overrides the whole case statement for rigid mountain bikes. 
Because Tm paranoid, I sometimes put a trap in the case statement: 

module MountainBike... 


def price 
case @type_code 
when : rigid 

raise "shouldn't get here" 
when :front_suspension 

(1 + ©commission) * ©base_price + ©front_suspension_price 
when :full_suspension 

(1 + ©commission) * ©base_price + ©front_suspension_price + 

©rear_suspensi on_pri ce 
end 
end 
end 
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All going well, the tests should pass. 
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I then do the same for the other type classes, removing the price method in the 
MountainBike module when I’m done. 

class RigidMountainBike 
include MountainBike 

def price 

(1 + @commission) * @base_price 
end 
end 

class FrontSuspensi onMountai nBi ke 
include MountainBike 

def price 

(1 + @commission) * @base_price + @front_suspension_price 
end 
end 

class Ful 1 Suspensi onMountai nBi ke 
include MountainBike 

def price 

(1 + ©commission) * @base_price + @front_suspension_price + 

@rear_suspensi on_pri ce 
end 
end 

module MountainBike. . . 
d e f pric e 
cas e ©typ e _cod e 

end 

end 

end 
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I can then do the same for off_road_ability. 

class RigidMountainBike 
include MountainBike 

def price 

(1 + ©commission) * @base_price 
end 

def off_road_ability 
@tire_width * TIRE_WIDTH_FACTOR 
end 
end 

class FrontSuspensi onMountai nBi ke 
include MountainBike 

def price 

(1 + ©commission) * ©base_price + ©front_suspension_price 
end 

def off_road_ability 

©tire_width * TIRE_WIDTH_FACTOR + ©front_fork_travel * 
FRONT_SUSPENSION_FACTOR 
end 
end 

cl ass Ful 1 Suspensi onMountai nBi ke 
include MountainBike 

def price 

(1 + ©commission) * ©base_price + ©front_suspension_price + 
©rear_suspensi on_pri ce 
end 

def off_road_ability 

©tire_width * TIRE_WIDTH_FACTOR + ©front_fork_travel * 
FRONT_SUSPENSION_FACTOR + ©rear_fork_travel * REAR_SUSPENSION_FACTOR 
end 
end 
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module MountainBike 
d e f off_road_abi1ity 
r e sult = @tir e _width - TIRE_WIDT H _FACTOR 

if @typ e _cod e = = :front_5usp e n5ion || (gtyp e _cod e == ifulLsusp e nsion 
r e sult += @front_fork_trav e 1 * FRONT_SUSPENSION_FACTOR 
end 

if @typ e _cod e == :fu11_susp e nsion 
r e sult + = @r e ar_fork_trav e 1 " REAR_SUSPENSION_FACTOR 
end 

r e sult 

end 

end 

Since we’re no longer using the type code, I can remove it from the callers. 

bike = FrontSuspensionMountainBike.new(:type_code => :front_suspension, 
:tire_width => 2, 

:front_fork_travel => 3) 

becomes 

bike = FrontSuspensionMountainBike.new(:tire_width => 2, :front_fork_travel => 3) 

We’ll keep the MountainBike module in this case since it still houses some useful 
code that would otherwise need to he duplicated. 


Replace Type Code ’with Module Extension 

You have a type code that affects the behavior of a class. 
Replace the type code with dynamic module extension. 


Mountain Bike 


type_code 

price q 




def price 

case @type_code 
when :rigid 

when :front_suspension 

when :full_suspension 

end 

end 
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Motivation 

Like Replace Type Code with Polymorphism, Replace Type Code with Mod- 
ule Extension aims to remove conditional logic. By extending a module, we 
can change the behavior of an object at runtime. Both the original class and 
the module that is being extended can access the same instance variables. This 
removes some of the headache that comes along with a more traditional state/ 
strategy pattern that uses delegation. 

The one catch with module extension is that modules cannot be unmixed 
easily. Once they are mixed into an object, their behavior is hard to remove. 
So use Replace Type Code with Module Extension when you don’t care about 
removing behavior. If you do care, use Replace Type Code with State/Strategy 
instead. 

Mechanics 

1. Perform Self-encapsulate Field on the type code. 

2. Create a module for each type code variant. 

3. Make the type code writer extend the type module appropriately. 

4. Choose one of the methods that use the type code. Override the method 
on one of the type modules. 

5. Test. 

6. Do the same for the other type modules. Modify the implementation on 
the class to return the default behavior. 

7. Test. 

8. Repeat for the other methods that use the type code. 

9. Test. 

10. Pass the module into the type code setter instead of the old type code. 
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Example 

We’ll use a similar example to Replace Type Code with Polymorphism. Let’s say 
we have a mountain hike object in the system that at some stage in its life cycle 
we decide to add front suspension to. 

bike = MountainBike.new(:type_code => : rigid) 

bike.type_code = :front_suspension 

MountainBike might look something like this: 
class MountainBike... 

attr_writer :type_code 

def initialize(params) 

@type_code = params[:type_code] 

@commission = params[: commissi on] 

end 

def off_road_ability 
result = @tire_width * TIRE_WIDTH_FACTOR 

if @type_code == :front_suspension || @type_code == :full_suspension 
result += @front_fork_trave1 * FRONT_SUSPENSION_FACTOR 
end 

if @type_code == :full_suspension 
result += @rear_fork_travel * REAR_SUSPENSION_FACTOR 
end 
result 
end 

def price 
case @type_code 
when : rigid 

(1 + ©commission) * @base_price 
when :front_suspension 

(1 + ©commission) * ©base_price + ©front_suspension_price 
when :full_suspension 

(1 + ©commission) * ©base_price + ©front_suspension_price + 
©rear_suspensi on_pri ce 
end 
end 
end 
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The first step is to use Self Encapsulate Field on the type code. I’ll create a 
custom attribute writer because it will do something more interesting soon, and 
call it from the constructor. 

class MountainBike. . . 

attr_reader :type_code 

def initial ize(params) 
self.type_code = params[:type_code] 

@commission = params[: commissi on] 

end 

def type_code= (value) 

@type_code = value 
end 

def off_road_ability 

result = @tire_width * TIRE_WIDTH_FACTOR 

if type_code == :front_suspension || type_code == :full_suspension 
result += @front_fork_travel * FRONT_SUSPENSION_FACTOR 
end 

if type_code == : full .suspension 
result += @rear_fork_travel * REAR_SUSPENSION_FACTOR 
end 

result 

end 

def price 
case type.code 
when : rigid 

(1 + ©commission) * ©base.price 
when :front_suspension 

(1 + ©commission) * ©base.price + ©front_suspension_price 
when :full_suspension 

(1 + ©commission) * ©base.price + ©front_suspension_price + 
©rear.suspensi on.pri ce 
end 
end 

The next step is to create a module for each of the types. We’ll make rigid the 
default, and add modules for FrontSuspensionMountainBike and FullSuspensionMountainBike. 
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I need to change the type code setter to extend the appropriate module. The 
case statement I’m introducing will be removed by the time we’ve finished this 
refactoring; it’s just there to make the next step a bit smaller. 

class MountainBike. . . 

def type_code=(value) 

@type_code = value 
case type_code 

when :front_suspension: extend(FrontSuspensionMountainBike) 
when :full_suspension: extend(FullSuspensionMountainBike) 
end 
end 

I then begin Replace Conditional with Polymorphism. I’ll start with price, and 
override it on FrontSuspensionMountainBike. 

module FrontSuspensi onMountai nBi ke 
def price 

(1 + ©commission) * @base_price + @front_suspension_price 
end 
end 

module Full Suspensi onMountai nBi ke 
end 

I can put a trap in the case statement to make sure it’s being overridden: 
class MountainBike... 
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def price 
case type_code 
when : rigid 

(1 + ©commission) * ©base_price 
when :front_suspension 
raise "shouldn't get here" 
when :full_suspension 

(1 + ©commission) * ©base_price + ©front_suspension_price + 
©rear_suspensi on_pri ce 
end 
end 
end 


At this point we haven’t gotten far, but our tests should pass. 
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I then repeat the process for FullSuspensionMountainBike. I can remove the case state- 
ment in price, and just return the default implementation for rigid mountain 
bikes. 

module FrontSuspensi onMountai nBi ke 
def price 

(1 + ©commission) * @base_price + @front_suspension_price 
end 
end 

module Ful 1 Suspensi onMountai nBi ke 
def price 

(1 + ©commission) * ©base_price + ©front_suspension_price + 

©rear_suspensi on_pri ce 
end 
end 

class MountainBike. . . 
def price 

(1 + ©commission) * ©base_price 
end 
end 

I then do the same for off_road_ability. The constants will have to be scoped to 
access them on MountainBike. 

module FrontSuspensi onMountai nBi ke 
def price 

(1 + ©commission) * ©base_price + ©front_suspension_price 
end 

def off_road_ability 

©tire_width * MountainBike: :TIRE_WIDTH_FACTOR + ©front_fork_trave1 * 
Mountai nBi ke : : FRONT_SUSPENSION_FACTOR 
end 
end 

module Ful 1 Suspensi onMountai nBi ke 
def price 

(1 + ©commission) * ©base_price + ©front_suspension_price + 

©rear_suspensi on_pri ce 
end 

def off_road_ability 

©tire_width * MountainBike: :TIRE_WIDTH_FACTOR + 

©front_fork_travel * MountainBike: :FRONT_SUSPENSION_FACTOR + 
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@rear_fork_travel * MountainBike: :REAR_SUSPENSION_FACTOR 
end 
end 

class MountainBike... 

def off_road_ability 
@tire_width * TIRE_WIDTH_FACTOR 
end 

def price 

(1 + ©commission) * @base_price 
end 
end 

I can remove the case statement I created by getting the callers to pass in the 
appropriate module. 

bike = MountainBike.new(:type_code => : rigid) 

bike.type_code = :front_suspension 
becomes 

bike = MountainBike. new 

bike.type_code = FrontSuspensionMountainBike 

class MountainBike... 

def type_code=(mod) 
extend(mod) 
end 

I should now have removed all traces of @type_code. 
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Replace Type Code with State/Strategy 

You have a type code that affects the behavior of a class and the type code 
changes at runtime. 

Replace the type code with a state object. 


Mountain Bike 

type_code 

price 


def price 

case @type_code 
when irigid 

when :front_suspension 

when :fuli_suspension 

end 

end 



Mountain Bike 


«protocol» 

bike type 

> 


price Q 

bikejype 

charge | 


@bike_type. price 




Front Suspension I 
Mountain Bike 

1 

Rigid Mountain 
Bike 



charge 

charge 


Front Suspension I 
Mountain Bike | 



charge I 


Motivation 

This refactoring has the same goal as Replace Type Code with Polymorphism 
and Replace Type Code with Module Extension: removing conditional logic. 
I use Replace Type Code with State/Strategy when the type code is changed at 
runtime and the type changes are complex enough that I can’t get away with 
Module Extension. 

State and strategy are similar, and the refactoring is the same whichever you 
use. Choose the pattern that better fits the specific circumstances. If you are try- 
ing to simplify a single algorithm, strategy is the better term. If you are going to 
move state-specific data and you think of the object as changing state, use the 
state pattern. 

Mechanics 
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1. Perform Self-encapsulate Eield on the type code. 
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2. Create empty classes for each of the polymorphic objects. Create a new 
instance variable to represent the type. This will be the object that we’ll 
delegate to. 

3. Use the old type code to determine which of the new type classes should 
be assigned to the type instance variable. 

4. Choose one of the methods that you want to behave polymorphically. 

Add a method with the same name on one of the new type classes, and 
delegate to it from the parent object 

You’ll need to pass in any state that needs to be shared with the 
object being delegated to, or pass a reference to the original object. 

5. Test. 

6. Repeat step 4 for the other type classes. 

7. Test. 

8. Repeat steps 4 through 7 for each of the other methods that use the type 
code. 

Example 

To easily demonstrate the differences between Replace Type Code with State/ 
Strategy and the other Replace Type Code refactorings, we’ll use a similar 
example. This time we’ll add methods for upgrading the mountain bike to add 
front suspension and rear suspension. 

class MountainBike. . . 

def initialize(params) 
set_state_f rom_hash (paraitis) 
end 

def add_front_suspensi on (paraitis) 

@type_code = :front_suspension 
set_state_from_hash (paraitis) 
end 

def add_rear_suspension(paranis) 
unless @type_code == :front_suspension 
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raise "You can't add rear suspension unless you have front suspension" 
end 

@type_code = :full_suspension 
set_state_f rom_hash (params) 
end 

def off_road_ability 

result = @tire_width * TIRE_WIDTH_FACTOR 

if @type_code == :front_suspension || @type_code == :full_suspension 
result += @front_fork_travel * FRONT_SUSPENSION_FACTOR 
end 

if @type_code == :full_suspension 
result += @rear_fork_travel * REAR_SUSPENSION_FACTOR 
end 

result 

end 


def price 
case @type_code 
when : rigid 

(1 + @commission) * @base_price 
when :front_suspension 

(1 + ©commission) * @base_price + @front_suspension_price 
when :full_suspension 

(1 + ©commission) * ©base_price + ©front_suspension_price + 

©rear_suspensi on_pri ce 
end 
end 

private 

def set_state_from_hash(hash) 

©base_price = hash[:base_price] if hash.has_key?(:base_price) 
i f hash . has_key? ( : f ront_suspensi on_pri ce) 
©front_suspension_price = hash[:front_suspension_price] 
end 

if hash.has_key?(: rear_suspension_price) 

©rear_suspension_price = hash[: rear_suspension_price] 
end 

if hash. has_key?(: commissi on) 

©commission = hash [: commissi on] 
end 

if hash.has_key?(:tire_width) 

©tire_width = hash[:tire_width] 
end 

i f hash . has_key? ( : front_fork_t ravel ) 
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@front_fork_travel = hash[:front_fork_travel] 
end 

i f hash . has_key? ( : rear_fork_travel ) 

@rear_fork_travel = hash[:rear_fork_travel] 
end 

@type_code = hash[:type_code] if hash.has_key?(:type_code) 
end 
end 

The first step is to perform Self-encapsulate Field on the type code. By confin- 
ing all access to the type code to just the getter and setter, we can more easily 
perform parallel tasks when the type is being accessed. This enables us to take 
smaller steps during the refactoring. 

We add an attr_reader, and a custom attribute writer, which will do something 
more interesting soon. 

class MountainBike. . . 
attr_reader :type_code 

def initialize(params) 
set_state_f rom_hash (parains) 
end 

def type_code=(value) 

@type_code = value 
end 

def add_front_suspension(params) 
self.type_code = :front_suspension 
set_state_f rom_hash (params) 
end 
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def add_rear_suspensi on (params) 
unless type_code == :front_suspension 
raise "You can't add rear suspension unless you have front suspension" 
end 

self.type_code = :full_suspension 
set_state_from_hash (params) 
end 

def off_road_ability 
result = @tire_width * TIRE_WIDTH_FACTOR 

if type_code == :front_suspension || type_code == :full_suspension 
result += @front_fork_travel * FRONT_SUSPENSION_FACTOR 
end 
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if type_code == : full .suspension 
result += @rear_fork_travel * REAR_SUSPENSION_FACTOR 
end 

result 

end 

def price 
case type.code 
when : rigid 

(1 + @commission) * @base_price 
when :front_suspension 

(1 + ©commission) * @base_price + @front_suspension_price 
when :full_suspension 

(1 + ©commission) * ©base.price + ©front_suspension_price + 

©rear.suspensi on.pri ce 
end 
end 

private 

def set_state_from_hash(hash) 

©base.price = hash[:base_price] if hash.has_key?(:base_price) 
i f hash . has.key? ( : f ront.suspensi on.pri ce) 
©front_suspension_price = hash[:front_suspension_price] 
end 

if hash.has_key?(: rear_suspension_price) 

©rear_suspension_price = hash[: rear_suspension_price] 
end 

if hash. has_key?(: commissi on) 

©commission = hash [: commissi on] 
end 

if hash.has_key?(:tire_width) 

©tire.width = hash[:tire_width] 
end 

i f hash . has.key? ( : front_fork_t ravel ) 

©front_fork_travel = hash[:front_fork_travel] 
end 

if hash.has_key?(: rear_fork_travel) 

©rear_fork_travel = hash[:rear_fork_travel] 
end 

self.type.code = hash[:type_code] if hash.has_key?(:type_code) 
end 
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Next we create empty classes for each variant of type code, 
class RigidMountainBike 
end 

class FrontSuspensi onMountai nBi ke 
end 

class Ful 1 Suspensi onMountai nBi ke 
end 

We need a new instance variable to represent the type. We’ll assign to it an 
instance of one of our new classes. This will be the object that we’ll delegate to. 

class MountainBike. . . 

def type_code=(value) 

@type_code = value 
@bike_type = case type_code 
when : rigid: RigidMountainBike. new 
when :front_suspension: FrontSuspensi onMountai nBi ke. new 
when :full_suspension: Full Suspensi onMountai nBi ke. new 
end 
end 


end 


Replace Type 
Code with 
State/ 
Strategy 


It may seem strange that we’re introducing a case statement when the pur- 
pose of this refactoring is, in fact, to remove conditional logic. The case state- 
ment just enables a smaller step to the refactoring, by giving us the ability to 
modify the internals of the class without modifying the callers. Rest assured that 
it won’t last long: By the time we finish the refactoring all conditional logic will 
be removed. 

Now the fun begins. We want to use Replace Conditional with Polymor- 
phism on the conditional logic. I’ll start with off_road_ability, and add it to 
RigidMountainBike. We have to pass in the instance variables that are needed by the 
state object. 

class RigidMountainBike... 

def initialize(parains) 

@tire_width = parains[:tire_width] 
end 


def off_road_ability 

@tire_width * MountainBike: :TIRE_WIDTH_FACTOR 
end 
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end 

class FrontSuspensi onMountai nBi ke 
end 

cl ass Ful 1 Suspensi onMountai nBi ke 
end 

class MountainBike. . . 


def type_code= (value) 

@type_code = value 
@bike_type = case type_code 

when :rigid: RigidMountainBike.new(:tire_width => @tire_width) 
when :front_suspension: FrontSuspensi onMountai nBi ke. new 
when :full_suspension: FullSuspensionMountainBike.new 
end 
end 


def off_road_ability 

return @bike_type.off_road_ability if type_code == : rigid 

result = @tire_width * TIRE_WIDTH_FACTOR 

if type_code == :front_suspension || type_code == :full_suspension 
result += @front_fork_travel * FRONT_SUSPENSION_FACTOR 
end 

if type_code == : full .suspension 
result += @rear_fork_travel * REAR_SUSPENSION_FACTOR 
end 

result 

end 


end 

At this stage we shouldn’t have broken anything. 
We do the same with the other new classes. 

class FrontSuspensi onMountai nBi ke 

def initial ize(params) 

@tire_width = params[:tire_width] 
@front_fork_travel = params[:front_fork_travel] 
end 


Replace Type 
Code with 
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def off_road_ability 

@tire_width * MountainBike: :TIRE_WIDTH_FACTOR + @front_fork_travel * 
Mountai nBi ke : : FRONT_SUSPENSION_FACTOR 
end 

end 

class Ful 1 Suspensi onMountai nBi ke 

def initialize(params) 

@tire_width = parains[:tire_width] 

@front_fork_travel = params[:front_fork_travel] 

@rear_fork_trave1 = params[:rear_fork_travel] 
end 

def off_road_ability 

@tire_width * MountainBike: :TIRE_WIDTH_FACTOR + 

@front_fork_travel * MountainBike: :FRONT_SUSPENSION_FACTOR + 
@rear_fork_trave1 * MountainBike: :REAR_SUSPENSION_FACTOR 
end 

end 

We can then make off_road_ability in mountain bike delegate to the type using 
Forwardable. (See the Hide Delegate section in Chapter 7 for an explanation of 
Forwardable.) 

class MountainBike... 


extend Forwardable 

def_delegators :@bike_type, :off_road_ability 
attr_reader :type_code 


Replace Type 
Code with 
State/ 
Strategy 


def type_code=(value) 

@type_code = value 
@bike_type = case type_code 

when : rigid: RigidMountainBike.new(:tire_width => @tire_width) 

when :front_suspension: FrontSuspensi onMountai nBi ke.newf 
:tire_width => @tire_width, 

:front_fork_travel => @front_fork_travel 

) 

when :full_suspension: Full Suspensi onMountai nBi ke.newf 
:tire_width => @tire_width, 

:front_fork_travel => @front_fork_travel , 

: rear_fork_travel => @rear_fork_travel 

) 
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end 

Our add_front_suspension and add_rear_suspension methods need to change to set the 
type object to an instance of one of our new classes. 

class MountainBike. . . 

def add_front_suspension(params) 
self.type_code = :front_suspension 
@bike_type = FrontSuspensionMountainBike.newf 
{ :tire_width => @tire_width}.merge(params) 

) 

set_state_from_hash(params) 

end 

def add_rear_suspension(params) 
unless type_code == :front_suspension 
raise "You can't add rear suspension unless you have front suspension" 
end 

self.type_code = :ful1_suspension 
@bike_type = Ful1SuspensionMountainBike.new({ 

:tire_width => @tire_width, 

:front_fork_travel => @front_fork_travel 
}.merge(params)) 
set_state_from_hash(params) 
end 
end 


We can then do the same with the price method, 
class RigidMountainBike. . . 

def price 

(1 + @commission) * @base_price 
end 

end 

class FrontSuspensionMountainBike. . . 
def price 

(1 + ©commission) * @base_price + @front_suspension_price 
end 


Replace Type 
Code with 
State/ 
Strategy 


end 
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class FullSuspensionMountainBike. . . 
def price 

(1 + ©commission) * @base_price + @front_suspension_price + 

@rear_suspensi on_pri ce 
end 

end 

Since price is the last method that we need to move, we can start to remove 
type_code. 

class MountainBike. . . 

def_delegators :@bike_type, :off_road_ability, : price 

def type_code=(value) 

@type_code = value 
@bike_type = case type_code 
when : rigid: RigidMountainBike.new( 

:tire_width => @tire_width, 

:base_price => @base_price, 

: commissi on => ©commission 

) 

when :front_suspension: FrontSuspensionMountainBike.new( 

:tire_width => ©tire_width, 

:front_fork_travel => ©f ront_fork_travel , 

:front_suspension_price => ©front_suspension_price, 

:base_price => ©base_price, 

: commissi on => ©commission 

) 

when :full_suspension: FullSuspensionMountainBike.new( 

:tire_width => ©tire_width, 

:front_fork_travel => ©f ront_fork_travel , 

:rear_fork_travel => ©rear_fork_travel , 

:front_suspension_price => ©front_suspension_price, 
:rear_suspension_price => ©rear_suspension_price, 

:base_price => ©base_price, 

: commissi on => ©commission 

) 

end 


end 
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def add_front_suspension(params) 
s e 1f.typ e _cod e = :front_susp e nsion 
@bike_type = FrontSuspensionMountainBike.new({ 
:tire_width => @bike_type.tire_width, 
:base_price => @bike_type.base_price, 

: commission => @bike_type. commissi on 
}.merge(params)) 
end 


def add_rear_suspension(params) 
uni ess @bi ke_type . i s_a?(FrontSuspensi onMountai nBi ke) 
raise "You can't add rear suspension unless you have front suspension" 
end 

s e 1f.typ e _cod e = :fu11_susp e nsion 
@bike_type = FullSuspensi onMountai nBi ke.new({ 

:tire_width => @bike_type.tire_width, 

:front_fork_travel => @bike_type.front_fork_travel , 

: f ront_suspensi on_pri ce => @bi ke_type . f ront_suspensi on_pri ce , 

:base_price => @bike_type.base_price, 

: commission => @bike_type. commissi on 
}.merge(params)) 
end 

Then we can change the callers of MountainBike and remove the case statement from 
our initialize method. 


bike = MountainBike.newf 
:type => :front_suspension, 

:tire_width => @tire_width, 

:front_fork_travel => @front_fork_travel , 
:front_suspension_price => @front_suspension_price, 
:base_price => @base_price, 

: commissi on => ©commission 

becomes 

bike = Mountai nBi ke.new(FrontSuspensi onMountai nBi ke.newf 
:typ e = > :front_susp e nsion , 

:tire_width => @tire_width, 

:front_fork_travel => @front_fork_travel , 
:front_suspension_price => @front_suspension_price, 
:base_price => @base_price, 

: commissi on => ©commission 
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And with most of the instance data being set directly on the type object, we 
can remove all that instance variable setting in MountainBike. 

class MountainBike... 
def initialize(bike_type) 
s e t_stat e _froin_hash(paraiiis) 

@bike_type = bike_type 
end 

d e f s e t_stat e _from_hash(hash) 

@bas e _pric e = hash[:bas e _pric e ] if hash.has_k e y?(:bas e _pric e ) 
i f hash . has_k e y? ( : f ront_susp e nsi on_pri c e ) 

@front_susp e nsion_pric e = hash[:front_susp e nsion_pric e ] 
end 

i f hash . has_k e y?( : r e ar_susp e nsi on_pri c e ) 

@r e ar_susp e nsion_pric e = hash[: r e ar_susp e nsion_pric e ] 
end 

if hash.has_k e y?(:coimission) 

(gcofflfflission = hash[:coimission] 
end 

@tir e _width = hash[:tir e _width] if hash.has_k e y?(:tir e _width) 
i f hash . has_k e y? ( : front_fork_t rav e l ) 

@front_fork_trav e 1 = hash[:front_fork_trav e 1] 
end 

i f hash . has_k e y? ( : r e ar_fork_trav e 1 ) 

@rear_fork_trav e 1 = hash[:r e ar_fork_trav e 1] 
end 

se1f.type_cod e = hash[:typ e _cod e ] if hash.has_k e y?(:typ e _cod e ) 
end 
end 
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Rather than reach into the type object when we’re upgrading, we can use 
Extract Method to encapsulate the upgradable parameters. 

class RigidMountainBike. . . 

attr_r e ad e r :tir e _width, :front_fork_trav e l , :front_susp e nsion_pric e, 
:bas e _pric e , :coimission 

def upgradable_paraineters 

{ 

:tire_width => @tire_width, 

:base_price => @base_price, 

:coimission => @coimission 

} 

end 

end 
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class FrontSuspensionMountainBike. . . 
attr_r e ad e r :tir e _width, :front_fork_trav e 1 , :front_susp e nsion_pric e, 
:base_pric e , :comiiiission 
def upgradable_parameters 
{ 

:tire_width => @tire_width, 

:front_fork_travel => @front_fork_travel , 

:front_suspension_price => @front_suspension_price, 

:base_price => @base_price, 

: commission => ©commission 

} 

end 

end 

class MountainBike. . . 
def add_front_suspension(params) 

@bike_type = FrontSuspensionMountainBike.new( 

@bi ke_type . upgradable_parameters . merge(params) 

) 

end 

def add_rear_suspension(params) 
uni ess @bi ke_type . i s_a?(FrontSuspensi onMountai nBi ke) 
raise "You can't add rear suspension unless you have front suspension" 
end 

@bike_type = FullSuspensi onMountai nBi ke.new( 

@bi ke_type . upgradable_parameters . merge(params) 

) 

end 

end 

We could then use Extract Module to remove any duplication in our new 
classes. 


Replace Subclass with Fields 

You have subclasses that vary only in methods that return constant data. 
Change the methods to superclass fields and eliminate the subclasses. 


Replace 
Subclass 
with Fields 


